Skip to content

Conversation

@ShaneIsley
Copy link
Owner

  • Add classify_error() to detect configuration issues (quota, auth, model)
  • Sequential runs abort early on fatal errors with clear guidance
  • Parallel runs report fatal errors after completion with suggestions
  • Add FATAL_ERROR_PATTERNS for common API errors
  • Add 5 tests for error classification

Example output on quota error:
❌ Fatal error detected: insufficient_quota 💡 API quota exceeded. Add credits at https://platform.openai.com/... Aborting benchmark - fix the issue above and retry.

This prevents wasting time running remaining samples when the first one fails due to a configuration issue that requires user action.

- Add classify_error() to detect configuration issues (quota, auth, model)
- Sequential runs abort early on fatal errors with clear guidance
- Parallel runs report fatal errors after completion with suggestions
- Add FATAL_ERROR_PATTERNS for common API errors
- Add 5 tests for error classification

Example output on quota error:
  ❌ Fatal error detected: insufficient_quota
  💡 API quota exceeded. Add credits at https://platform.openai.com/...
  Aborting benchmark - fix the issue above and retry.

This prevents wasting time running remaining samples when the first one
fails due to a configuration issue that requires user action.
@ShaneIsley ShaneIsley merged commit 0a3739c into main Jan 17, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants